Changed to update listings model to fix the listings update issue and… - #90
Merged
Alexandre Zollinger Chohfi (azchohfi) merged 1 commit intoSep 12, 2025
Merged
Conversation
… honour all the fields
There was a problem hiding this comment.
Pull Request Overview
This PR fixes the listings update issue by aligning the model structure with Win32 API expectations and adds missing fields to support comprehensive listing updates.
- Changed the Listings property from a List to a single Listing object in UpdateMetadataRequest
- Added five new fields to the Listing model: WhatsNew, ShortDescription, Copyright, DevelopedBy, SortTitle, and ContactInfo
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| MSStore.API/Models/UpdateMetadataRequest.cs | Changed Listings property type from List to single Listing object |
| MSStore.API/Models/Listing.cs | Added six new string properties to support additional listing metadata fields |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Alexandre Zollinger Chohfi (azchohfi)
approved these changes
Sep 12, 2025
Alexandre Zollinger Chohfi (azchohfi)
merged commit Sep 12, 2025
417f38a
into
main
15 of 16 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed to update listings model to align it with the request that Win32 API expected. This will fix the listings update issue. Also added all the missing fields to support the listing update for other fields as well.